-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Scalable jenkins #7
base: master
Are you sure you want to change the base?
Conversation
4f57984
to
e609998
Compare
e609998
to
8d89444
Compare
description = "The tags to associate with your network and subnets." | ||
|
||
default = { | ||
environment = "Test" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we configure terraform to add a lock to the Jenkins master?
.withName("ubuntunonsgx") | ||
.withLabels("nonSGX") | ||
.withLocation(AGENTS_LOCATION) | ||
.withVirtualMachineSize(VM_SIZE_NONSGX) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't believe this sizing is correct, I believe we actually needed a "Standard_F8s_v2". This caused tls issues.
ae964d1
to
17bc73f
Compare
9ede09e
to
6852a8b
Compare
6852a8b
to
a2f3147
Compare
b46ff83
to
32fbdc1
Compare
f710929
to
74b960a
Compare
Signed-off-by: Marius Oprin <[email protected]>
74b960a
to
c5edccf
Compare
run integration testing |
@oprinmarius: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by opening an issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
It creates a VM in Azure and configures Jenkins to run in docker.